Skip to content

fix: simplify CLI API error output#120

Draft
Ramon Nogueira (ramon-langchain) wants to merge 1 commit into
mainfrom
ramonn/cli-api-error-display
Draft

fix: simplify CLI API error output#120
Ramon Nogueira (ramon-langchain) wants to merge 1 commit into
mainfrom
ramonn/cli-api-error-display

Conversation

@ramon-langchain
Copy link
Copy Markdown
Member

Summary

  • format LangSmith SDK API errors only at the CLI stderr boundary
  • preserve original wrapped errors from command execution
  • render validation details as concise messages such as 422 Unprocessable Entity: ...

Test Plan

  • go test ./internal/cmd -run 'TestFormatErrorMessage|TestSandboxCreateCmd_RendersAPIValidationError'
  • deslop cmd/langsmith/main.go internal/cmd/errors.go internal/cmd/errors_test.go internal/cmd/sandbox_test.go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant